Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade PyXForm dependency to v1.1.0 #1796

Merged
merged 9 commits into from
Mar 27, 2020
Merged

Upgrade PyXForm dependency to v1.1.0 #1796

merged 9 commits into from
Mar 27, 2020

Conversation

DavisRayM
Copy link
Contributor

@DavisRayM DavisRayM commented Mar 3, 2020

Changes / Features implemented

  • Upgrade pyxform to v1.1.0

Steps taken to verify this change does what is intended

  • Updated tests
  • Tested functionality locally

Pending:

  • Test that this change is backwards compatible

Side effects of implementing this change

Still researching, impacts of this change

Closes #1787

@DavisRayM DavisRayM changed the title Upgrade pyxform dependency to v1.1.0 Upgrade PyXForm dependency to v1.1.0 Mar 5, 2020
@DavisRayM DavisRayM marked this pull request as ready for review March 5, 2020 14:20
@DavisRayM DavisRayM force-pushed the pyxform-1.-upgrade branch 2 times, most recently from e0be485 to f45ab96 Compare March 27, 2020 06:01
Add MULTI_SELECT_TYPE and SELECT_BIND_TYPE tags removing the previously
used SELECT_ONE and SELECT_MULTIPLE variables. More info:
XLSForm/pyxform#410
Modify Export Builder tests passing in the new 'default_name' arguement
to PyXForms 'create_survey_from_xls' function and use SELECT_BIND_TYPE
and MULTIPLE_SELECT_TYPE common tags instead of static values. More info:
XLSForm/pyxform#375
Modify _get_sav_options function to work with the new string binding
for select one elements
Update the error raised when an id_string is invalid. PyXForm now
utilizes the filename as the form_id if it's not explicitly set in the
settings sheet. More Info: XLSForm/pyxform#389
As of PyXForm v1.0.0 the name of the file is no longer used as the root
node. More Info: XLSForm/pyxform#375
As of PyXForm v1.0.0 the 'list_name' field has been added to a forms
generated JSON. More Info: XLSForm/pyxform#357
Update the 'test_choice_duplicate_error' test to expect the correct
error message raised by PyXForm.
Update the test and add a docstring explaining when the error occurs
@faith-mutua
Copy link

@DavisRayM I have tested the following scenarios and they have passed QA.

  • User can publish a form
  • User can replace a form - both old and new forms
  • User can share a project
  • User can create a form using the form builder
  • User can edit forms created using the form builder
  • Uploading forms with a field name data rises the error There are two sections with the name data.
  • User can export data
  • User can upload form media files
  • User can upload XLS Report templates and generate reports.

@faith-mutua faith-mutua added the QA+ PR passed QA testing label Mar 27, 2020
@DavisRayM DavisRayM merged commit cdb5a95 into master Mar 27, 2020
@DavisRayM DavisRayM deleted the pyxform-1.-upgrade branch March 27, 2020 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QA+ PR passed QA testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Investigate requirement upgrades required for the Django 2.2 upgrade
3 participants